Skip to content

BDMS-388 Create a standard set of PointIDs that can be used to test the frontend#309

Merged
jirhiker merged 1 commit into
stagingfrom
kas-showcase-pointids
Dec 23, 2025
Merged

BDMS-388 Create a standard set of PointIDs that can be used to test the frontend#309
jirhiker merged 1 commit into
stagingfrom
kas-showcase-pointids

Conversation

@ksmuczynski

@ksmuczynski ksmuczynski commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

BDMS-388 Create a standard set of PointIDs that can be used to test the frontend

Why

This PR addresses the following problem / context:

  • This PR addresses the need to establish a consistent set of PointIDs that can be utilized for manual testing and demonstrations of frontend functionalities.

How

Implementation summary - the following was changed / added / removed:

  • A SQL query was written to select 5 random PointIDs from the NMAquifer legacy system that met the following criteria:

    • Location records must have SiteDate populated. The final dataset should contain 2 records with WL_Continuous = true and 3 records with WL_Continuous = false.
    • Owner info must have values for last name, first name, a physical address OR a mailing address , one phone number
    • WellData should have HoleDepth, WellDepth, CasingDiameter, Status, CurrentUse, and MonitoringStatus populated
    • WaterLevels must have at least 10 records and MeasuringAgency=NMBGMR

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Bullet points here

@ksmuczynski
Create showcase_pointids
54dcb52 
Create new text file listing a set of PointIDs that can be utilized for manual testing and demonstrations of frontend functionalities in Ocotillo.
@codecov-commenter

codecov-commenter commented Dec 16, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@jirhiker

jirhiker commented Dec 16, 2025

Copy link
Copy Markdown
Member

I really like the programmatic approach. Do all of these pointids also exist in Ocotillo?

@chasetmartin

chasetmartin commented Dec 16, 2025

Copy link
Copy Markdown
Collaborator

I really like the programmatic approach. Do all of these pointids also exist in Ocotillo?

Agreed. And yes I just looked they all exist in Ocotillo Staging

chasetmartin
chasetmartin approved these changes Dec 16, 2025
View reviewed changes

@chasetmartin chasetmartin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work. Hoping for confirmation from folks on whether this file should live in the repo and be merged in, or should we document these in a different spot?

@jacob-a-brown

jacob-a-brown commented Dec 17, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

While WL_Continuous is a field that should indicate the existence of continuous water levels for that well, it may not have been maintained; we should confirm in the continuous water level tables that they're there.

@jirhiker jirhiker merged commit 5162035 into staging Dec 23, 2025
6 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the kas-showcase-pointids branch February 26, 2026 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chasetmartin chasetmartin chasetmartin approved these changes
@jirhiker jirhiker Awaiting requested review from jirhiker
@jacob-a-brown jacob-a-brown Awaiting requested review from jacob-a-brown

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ksmuczynski @codecov-commenter @jirhiker @chasetmartin @jacob-a-brown